-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
systemd-boot: Revert boot counting again #334526
systemd-boot: Revert boot counting again #334526
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable at a skim. I hope we can get this back ASAP once the dust has settled and the channels are moving again. I will check tests on aarch64-linux
.
I don't have an intermediate use case for boot counting, so I probably won't spend time on getting it to work. This was only happening on aarch64, right? |
No, eval broke for the entire channel. |
And the specialisation bug was platform independent |
Now I don't need to carry around an old version of nixpkgs just for nextcloud 27 anymore. I mainly wanted to do this update because of boot counting, but that has since been reverted: NixOS/nixpkgs#334526
Description of changes
Unfortunately, with the specialisation-related regression caused by boot counting, and fixes like #334473 being generally hasty and unsatisfactory, the most effective way to unblock the unstable channel is to simply revert boot counting.
This was made a little tricky, as there were three, other, PRs that were merged for systemd-boot this week, which interfered with the revert. I believe I've preserved those PRs correctly, and all systemd-boot tests seem to pass, including the
specialisation
one on aarch64 (it's the only one testing thedevicetree
change).Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.